Skip to content

Conversation

CarlosChub27
Copy link
Contributor

When using the Tailwind CSS package in mobile mode, when pressing the button to display collapsed items, these items were not displayed correctly. This was due to an issue with the visibility of the items inside the collapsed container.

With the change implemented, I adjusted the visibility settings and behavior of the items when interacting with the button in mobile mode.

All Submissions:

  • Have you followed the guidelines in our Contributing document?
  • Have you checked to ensure there aren't other open Pull Requests for the same update/change?

Changes to Core Features:

  • Have you added an explanation of what your changes do and why you'd like us to include them?
  • Have you written new tests for your core changes, as applicable?
  • Have you successfully ran tests with your changes locally?

@lrljoe
Copy link
Collaborator

lrljoe commented Apr 1, 2025

Thanks, I will review this week, apologies but I've been on holiday

Copy link

codecov bot commented Apr 1, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 89.96%. Comparing base (d96df8e) to head (ac15391).
Report is 9 commits behind head on development.

Additional details and impacted files
@@              Coverage Diff               @@
##             development    #2228   +/-   ##
==============================================
  Coverage          89.96%   89.96%           
  Complexity          2154     2154           
==============================================
  Files                242      242           
  Lines               4853     4853           
==============================================
  Hits                4366     4366           
  Misses               487      487           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link

stale bot commented May 1, 2025

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the wontfix This will not be worked on label May 1, 2025
@lrljoe lrljoe merged commit 40887ee into rappasoft:development May 3, 2025
11 of 12 checks passed
@lrljoe lrljoe mentioned this pull request May 3, 2025
lrljoe added a commit that referenced this pull request May 3, 2025
* Link column fix (#2223)

* Add handling for From field for "LinkColumn"

* Fix styling

* Update ChangeLog

* docs: add reminder of requirement to use toggle method (#2208)

Co-authored-by: Joe <[email protected]>

* Update IsNumericFilter.php (#2230)

* Link column fix (#2223) (#2224)

* Link column fix (#2223)

* Add handling for From field for "LinkColumn"

* Update ChangeLog

* Update IsNumericFilter.php

Fixes issues 2229

---------

Co-authored-by: Joe <[email protected]>

* Fix styling

* Boolean filter for bootstrap (#2244)

* Add BooleanFilter option for Bootstrap

* Add BS4 Support

* Update docs

* Fixes for livewire component column (#2245)

* Initial Commit

* Fix styling

* Initial Commit

* Fix styling

* Adjust test

* Fix styling

* Use test-livewire-column-component

* Additional Tests for LivewireComponentColumn

* Fix styling

* Add getContents Test

* TypeHint/PHPDoc Tidying (#2246)

* Fix broken typehints

* Minor tweaks to Footer/SecondaryHeader to ensure presence of callback prior to executing

* Fix styling

* PHPDoc additions

* Fix styling

* Update ChangeLog for TypeHintFixes

* Fix styling

* FilterPillData and StandardFilterPillData phpdocs

* Fix styling

* Add Wrapper Options to ArrayColumn (#2255)

* Add Wrapper Options to FlexCol

* Fix styling

* Moving getContents Back

* Fix styling

* Add Docs

* Add Wrapper Manual Tests

* Fix styling

* Update to test separator

* Fix styling

* Improved pagination UX (#2251)

* Link column fix (#2223) (#2224)

* Link column fix (#2223)

* Add handling for From field for "LinkColumn"

* Update ChangeLog

* Improved pagination UX

---------

Co-authored-by: Joe <[email protected]>

* Fixed collapsed function bug with tailwind CSS on mobile version (#2228)

* v3.7.2 updates (#2256)

---------

Co-authored-by: da-mask <[email protected]>
Co-authored-by: G4Zz0L1 <[email protected]>
Co-authored-by: Daniel Škopek <[email protected]>
Co-authored-by: CarlosChub27 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

wontfix This will not be worked on

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants